Skip to content

Conversation

@tomballgithub
Copy link
Contributor

@tomballgithub tomballgithub commented Jan 22, 2026

When activity is returned, it is causing an error because it is a dict not a list.

This fixes: #85

@github-actions github-actions bot added the 👻 bug Something isn't working label Jan 22, 2026
@codecov
Copy link

codecov bot commented Jan 22, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.34%. Comparing base (c43ab4d) to head (aa18143).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #86   +/-   ##
=======================================
  Coverage   88.34%   88.34%           
=======================================
  Files          54       54           
  Lines        2857     2857           
=======================================
  Hits         2524     2524           
  Misses        333      333           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Owner

@tr4nt0r tr4nt0r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thank you. But could you please also update the tests? I think it should be enough to replace presence.json with one of your examples that you have captured

@tr4nt0r tr4nt0r marked this pull request as draft January 22, 2026 08:19
@github-actions github-actions bot added the 🧪 testing Pull request that adds tests label Jan 22, 2026
⬆️ Bump release-drafter/release-drafter digest to 6db134d (tr4nt0r#87)
@tomballgithub
Copy link
Contributor Author

tomballgithub commented Jan 22, 2026

I added a new test and verified it passes with this PR and fails without it

@tr4nt0r
Copy link
Owner

tr4nt0r commented Jan 22, 2026

can you put the test data in a JSON fixture in tests/data/responses ?

@tomballgithub
Copy link
Contributor Author

I'll figure it out

@tomballgithub
Copy link
Contributor Author

I made the requested change. It passes all my checks

@tr4nt0r tr4nt0r removed the 🧪 testing Pull request that adds tests label Jan 22, 2026
@tr4nt0r tr4nt0r marked this pull request as ready for review January 22, 2026 20:14
Copy link
Owner

@tr4nt0r tr4nt0r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thank you 👍🏼

@tr4nt0r tr4nt0r merged commit 0f6c286 into tr4nt0r:main Jan 22, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

👻 bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Validation errors for PresenceItem

2 participants